-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Prettier formatting #168
base: master
Are you sure you want to change the base?
Conversation
5ac269a
to
d2155ff
Compare
# Please run `pre-commit run --all-files` when adding or changing entries. | ||
|
||
repos: | ||
- repo: https://github.com/pre-commit/mirrors-prettier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you prefer installing from https://github.com/pre-commit/mirrors-prettier or using the Nix package?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm completely unfamiliar with that, so I'd have to look into it. Unfortunately devenv isn't even in nixpkgs, so this will require a fair bit more setup. If someone else is up for converting this to devenv, feel free!
I've split off two commits which are only tangentially related to separate PRs.
Ping? |
db61020
to
9d683a7
Compare
Ping? It's a bit of work rebasing this… |
9d683a7
to
1f1049f
Compare
1f1049f
to
92bb297
Compare
Rebase ping… |
Actual command: `nix-shell --packages nodePackages.prettier --pure --run 'prettier --write .'`.
92bb297
to
84ef05d
Compare
Part of #167.